Skip to content

Hw6#10

Open
NosovDima wants to merge 9 commits into
masterfrom
hw6
Open

Hw6#10
NosovDima wants to merge 9 commits into
masterfrom
hw6

Conversation

@NosovDima
Copy link
Copy Markdown
Owner

Добрый вечер!
Проверьте пожалуйста дз

Copy link
Copy Markdown

@mielilon mielilon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Подводя итог, могу сказать, что работа выполнена хорошо. Рада видеть реализацию дополнительных заданий.

Предлагаю доработать 4-ю задачу и отправить код на повторную проверку.

Comment thread hw6script.js
const numbers = [1, 5, 4, 10, 0, 3];

for (let i = 0; i < numbers.length; i++) {
if (numbers[i] <= 10) break;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

По итогу мы получим undefined, потому что в условии происходит сравнение (меньше или равно). Нам нужно проверить на строгое равенство с 10, правильно будет так:
if (numbers[i] === 10) break;

Также нужно поменять условие и вывод через alert местами, так как по условию нужно вывести 10 в том числе.

Comment thread hw6script.js Outdated
numbers = numbers.join(" ");
alert(numbers);
// Exersice 4
let array = [
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Выполнено не совсем корректно. Так должен выглядеть итоговый результат, начальное значение – пустой массив []. За счет вложенных циклов нужно реализовать решение.

Comment thread hw6script.js
let search = numbers.includes(num);
alert(numbers.includes(num));
// Exersice 8
let letters = "abcdef";
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Решено задание верно, однако можно выполнить сокращение и сгруппировать методы.
Получится так:
let line = 'abcdef'; let lineReverse = line.split('').reverse().join(''); console.log(lineReverse);

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants